Day 28 - Review code

因團隊制宜。
假設桌球王成立團隊，將會有以下Pricinple、SOP、CheckList、Take away。

Principle

以Priority由上而下:

0. 確認邏輯無誤
1. 確認如何操作APP
2. 了解背後運作邏輯
3. 減少重複
   : 重複使用的邏輯包成function
   : 不用改一個地方，四處都要改
4. 增加可讀
   : 命名
   : 結構
   : 刪除沒有使用的

SOP & Document

可執行 round N ，Ｎ大於等於1。

round N ，依複雜性，及當round吸收程度來決定N為多少。
每 round 有其主題，比如說某次Release，加了哪些Feature、操作步驟/流程、前端架構，後端API、網路連線(關聯系統)、Config.等等。

最後的吸收理解可以邊看文件邊Trace Code，整個走一次確認是否大略知道重點，以便未來查案或加值。

Check List

• [ ] 了解user怎麼操作，通常哪些使用頻繁
• [ ] 前後端交握，哪些前端處理，傳遞給後端，後端Status code 回傳
• [ ] 上下游(關聯) 系統，如何溝通，正常異常判別
• [ ] log 識別 以及 量的控制

Take away

Review Code 的同時其實可以看到更多或是更好的寫法或架構，

如果可以看出原開發沒有發現的邏輯問題，就能即時在上線前排除，
也能想想看有沒機會在不改變輸出的情況，改出易於維運的Code。

其實是 reviewer 與 commiter(developer) 之間的切磋成長!

相關參考

• Code Review 怎麼做